Skip to content

Add Cockpit bulk process instance delete#65

Open
hauptmedia wants to merge 1 commit into
mainfrom
codex/cockpit-bulk-instance-delete
Open

Add Cockpit bulk process instance delete#65
hauptmedia wants to merge 1 commit into
mainfrom
codex/cockpit-bulk-instance-delete

Conversation

@hauptmedia

Copy link
Copy Markdown

Summary

  • add an async process-instance delete API wrapper and state signal for POST /process-instance/delete
  • add live-instance selection and confirmation controls in the process definition instances tab
  • include delete reason, skip listener/subprocess/IO mapping flags, batch result link, i18n, CSS, and tests

Tests

  • node -e "JSON.parse(require('fs').readFileSync('public/locales/en-US/translation.json','utf8')); JSON.parse(require('fs').readFileSync('public/locales/de-DE/translation.json','utf8'));"
  • npx eslint src/pages/Processes.jsx src/pages/Processes.test.jsx src/api/resources/process_instance.js src/api/resources/process_instance.test.js src/state.js src/state.test.js
  • npx vitest run src/api/resources/process_instance.test.js src/state.test.js src/pages/Processes.test.jsx
  • npx vitest run --maxWorkers=4
  • npm run build
  • Browser smoke: http://127.0.0.1:5176/processes/proc:1/instances reached the login boundary with no console errors

Local note

  • npx vitest run without worker limiting reported Vitest fork-runner timeouts locally after all 447 tests had passed; the constrained full-suite run above passed cleanly.

@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant